Skip to content

Conversation

fabiojaniolima
Copy link

This bulk assignment approach avoids typing errors. If the property exists the value will be assigned.

Copy link

@amims71 amims71 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how about checking if the $data array has any value out of the if-else list? could be a security bridge!

@fabiojaniolima
Copy link
Author

@amims71

I understand. I had thought of it this way to make the input more dynamic.

I added a list of allowed inputs in my last comia.

@fabiojaniolima
Copy link
Author

@amims71 I admit I didn't understand why Travis.ci failed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants